-
Notifications
You must be signed in to change notification settings - Fork 154
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix: if field has custom decoder, schema takes it into account #462
Conversation
Merge from original to update fokr
Pull from master repository
Pull from main repo
… as it will be replaced by it
This is just an initial version. I'd like to know if I'm in the right direction or if this isn't it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for contribution!
Minor comments, I don't think the approach is wrong, but I'm also not very strong on marshmellow side, so a bit of explanation of this solution would be nice :)
@george-zubrienko I reverted all formatting changes except for a single line which only contained spaces which PyCharm just won't let me change back, I hope it's not a deal breaker. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, nice and clear! Thanks a lot for fixing this bug!
Fix: if field has custom decoder, schema takes it into account (lidatong#462)
If a field has a custom decoder, replace its _deserialize logic with a dummy method, as it will be taken care of by the decoder.
Attempts to solve #460